-
-
Notifications
You must be signed in to change notification settings - Fork 109
🆕 Add CoderDojo 播磨科学公園都市 in 兵庫県 #1723
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
対応ありがとうございます!
コミットの整理をしていただけたら、それ以外は良さそうです🙆♀️
db/dojos.yaml
Outdated
name: 播磨科学公園都市 | ||
prefecture_id: 28 | ||
logo: "/img/dojos/default.webp" | ||
url: https://codeclub.org/ja/clubs/1700a138-2909-4e4e-9e75-88ebcbbbc55a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
url: https://codeclub.org/ja/clubs/1700a138-2909-4e4e-9e75-88ebcbbbc55a
@nacchan99 @rakuda-san-desu こちらリンク先にある活動内容を自分の方で確認してから対応した方が良さそうでしたので、マージするかどうかは自分の方で一旦引き取りますね! PR のご対応ありがとうございました...!! 🙏✨
8e33cb0
to
e9045f1
Compare
@yasulab 不要なコミットをカットしました & approved しました! |
並行して複数のPRがDojoを追加する際に、同じIDを使用してしまう 問題を事前に検出するためのテストを追加。 これにより、マージ前にCI/CDで重複IDを自動検出できるようになる。
PR #1738で砧(ID: 342)が先にマージされたため、 播磨科学公園都市のIDを342から343に変更。 並行作業による重複ID問題を解決。
諸々の対応が完了したので(細かなテスト追加、重複 ID の対処なども対応したので)、こちらマージしますね!ご対応ありがとうございました...!! (≧∇≦)b✨ |
やったこと